test(convert): add tests for convert_to_json - #711
Conversation
|
Thanks for the PR, @GonzaFP! We'll try to get to the review soon :) |
|
Quick check in here, @GonzaFP 👋 Do we need to add all of these tests, or are these functionalities being covered by Happy to keep these changes if they're needed, but it'd be great if you could verify if some of the tests are repeated :) |
|
Thanks @andrewtavis for your response. Should I combine all three tests into one? |
|
@GonzaFP: Note that I do still have this PR in mind. I've been away over the weekend. The community will be switching over to Scribe-Data from Scribe-Android this coming weekend and we'll bring this in then 😊 Sorry for your wait! |
|
Noted @andrewtavis |
|
Thank you @andrewtavis. |
Contributor checklist
pytestcommand as directed in the testing section of the contributing guideDescription
Added tests for
convert_to_jsonintest/cli/convert/wrapper.pyto improve test coverage.Related issue